Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Aggregate ctl-npm-meta package for NPM deps #1666

Merged
merged 5 commits into from
Jan 22, 2025
Merged

Aggregate ctl-npm-meta package for NPM deps #1666

merged 5 commits into from
Jan 22, 2025

Conversation

errfrom
Copy link
Collaborator

@errfrom errfrom commented Jan 15, 2025 •
edited
Loading

Closes Create an aggregate NPM package for CTL dependencies #1659

New repository: https://github.com/mlabs-haskell/ctl-npm-meta
New npm package: https://www.npmjs.com/package/@mlabs-haskell/ctl-npm-meta

TODO

Pre-review checklist

  • All code has been formatted using our config (make format)
  • Any new API features or modification of existing behavior are covered with tests
  • The template (templates/ctl-scaffold) has been updated
  • The changelog has been updated under the ## Unreleased header, using the appropriate sub-headings (### Added, ### Changed, ### Removed, ### Fixed), and the links to the appropriate issues/PRs have been included

@errfrom errfrom changed the title Switch to aggregate ctl-npm-meta package to Aggregate ctl-npm-meta package for NPM deps Jan 15, 2025
@errfrom errfrom self-assigned this Jan 15, 2025
@errfrom errfrom force-pushed the dshuiski/npm-meta-package branch from b4ac553 to f98a2ed Compare January 15, 2025 15:02
@errfrom errfrom force-pushed the dshuiski/npm-meta-package branch from 73a83e5 to 01bb8be Compare January 15, 2025 17:01
@errfrom errfrom marked this pull request as ready for review January 15, 2025 17:15
@errfrom errfrom merged commit 493bbd0 into develop Jan 22, 2025
3 checks passed
@errfrom errfrom deleted the dshuiski/npm-meta-package branch January 22, 2025 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@KristianBalaj KristianBalaj KristianBalaj approved these changes

@marcusbfs marcusbfs Awaiting requested review from marcusbfs

Assignees

@errfrom errfrom

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create an aggregate NPM package for CTL dependencies
2 participants
@errfrom @KristianBalaj